Codecov Report

Attention: Patch coverage is 0% with 255 lines in your changes missing coverage. Please review.

Project coverage is 90.3%. Comparing base (1125579) to head (baeb7f8).
Report is 39 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (1125579) and HEAD (baeb7f8). Click for more details.

HEAD has 133 uploads less than BASE

Flag	BASE (1125579)	HEAD (baeb7f8)
	151	18

@@           Coverage Diff           @@
##            main    #852     +/-   ##
=======================================
- Coverage   95.6%   90.3%   -5.4%     
=======================================
  Files         46      47      +1     
  Lines       4333    4588    +255     
=======================================
  Hits        4143    4143             
- Misses       190     445    +255     

In the MESSAGE meeting just now, @behnam-zakeri mentioned that #680 might be related.

@glatterf42 Related to the test, I have added an example of Westeros model via the Excel file, which is used by the code to create a Westeros scenario with time slices, exactly similar to the one we have in our published tutorials (Westeros_seasonality.ipynb). So, the only part missing will be to check if the output of these two models are the same or not.

1. Running the new code with input data from Westeros_baseline scenario and config data from Excel. [done]
2. Loading an already existing or running Westeros_Seasonality scenario (can be done easily).
3. Comparing the results of 1 and 2, e.g.:

• asserting if "time" related sets are the same ("time", "duration_time")
• asserting OBJ.
• asserting ACT of coal_ppl or wind_ppl in different times (winter or summer).

Unfortunately, neither @behnam-zakeri nor me have enough time to finish this PR in time for the 3.10 milestone. We hope this will be better for 3.11, though.